Skip to content

Conversation

@Honglei-Qiu
Copy link
Contributor

PR Category

Feature Enhancement

Description

完成Repo GraphSample SubgraphSource DimensionGeneralizationSource表的插入,DataTypeGeneralizationSource暂时未操作,不知道具体格式

@paddle-bot
Copy link

paddle-bot bot commented Jan 29, 2026

Thanks for your contribution!

@@ -0,0 +1,81 @@
-- SQLite
-- create Repo table
DROP TABLE IF EXISTS Repo;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个很危险啊

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个脚本一旦重复执行,数据就会立刻丢掉

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

已修改

Comment on lines 21 to 22
repo_uid VARCHAR(255) NOT NULL,
relative_model_path TEXT NOT NULL UNIQUE,
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

relative_model_path单独应该不是unique的。(relative_model_path, repo_uid)合起来才是。

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

每个表的联合索引都需要小心处理,区分度大的放前面

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

已修改,现在在写ORM

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

重命名成create_main_tables_$(date +"%Y-%m-%d-%H%M%S").sql

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

已修改

@Honglei-Qiu
Copy link
Contributor Author

Honglei-Qiu commented Jan 30, 2026

BackwardGraphSource 后续再改进,而且这个要确定最后的结果是对子图进行的backward存储还是整图的backward也要存储

@lixinqi
Copy link
Collaborator

lixinqi commented Jan 30, 2026

BackwardGraphSource 后续再改进,而且这个要确定最后的结果是对子图进行的backward存储还是整图的backward也要存储

可以



# DimensionGeneralizationSource insert func
def insert_DimensionGeneralizationSource(
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这些函数名字太诡异了

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

insert_dimension_info ?
insert_datatype_info

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

insert_dimension_generalization_source

Copy link
Contributor Author

@Honglei-Qiu Honglei-Qiu Jan 31, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done,目前完成数据库的init和insert功能,下周继续完成数据库的delete功能,updata功能待定,同时添加“运行insert的sh功能”时产生时间戳log,表示在哪时候插入,init产生时间戳log,都加一个时间戳log,后续可以分析时间戳log来获得某些额外信息
这样可以替代create_main_tables_$(date +"%Y-%m-%d-%H%M%S").sql功能
示例:python ./sqlite/orm_models.py init | tee "sqlite/logs/init_$(date +'%Y-%m-%d-%H%M%S').log"

@lixinqi lixinqi merged commit 2335af3 into PaddlePaddle:develop Feb 3, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants